Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build statically linked binaries for kubernetes. #5479

Merged
merged 1 commit into from
Mar 16, 2015

Conversation

ArtfulCoder
Copy link
Contributor

Build statically linked binaries. With the change to go 1.4, we probably were generating dynamically linked libraries accidentally.

…ably were generating dynamically linked binaries accidentally.
@ArtfulCoder
Copy link
Contributor Author

@lavalamp @dchen1107 @thockin
PTAL

@lavalamp
Copy link
Member

LGTM

@thockin thockin added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 15, 2015
j3ffml added a commit that referenced this pull request Mar 16, 2015
Build statically linked binaries for kubernetes.
@j3ffml j3ffml merged commit 9df2ea4 into kubernetes:master Mar 16, 2015
@zmerlynn zmerlynn mentioned this pull request Mar 16, 2015
vmarmol added a commit to vmarmol/kubernetes that referenced this pull request Mar 16, 2015
We allow cgo when available since some of our server components (mainly
Kubelet) rely on it. The current driving factor for the static binaries
was building Docker images from scratch. We will instead use images
built from small busybox bases.

Fixes kubernetes#5517
zmerlynn added a commit that referenced this pull request Mar 16, 2015
Revert #5479, don't force no cgo.
akram pushed a commit to akram/kubernetes that referenced this pull request Apr 7, 2015
We allow cgo when available since some of our server components (mainly
Kubelet) rely on it. The current driving factor for the static binaries
was building Docker images from scratch. We will instead use images
built from small busybox bases.

Fixes kubernetes#5517
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants